Skip to content

Comments

Fix the conflict with iterator_tools.#77

Merged
Luthaf merged 1 commit intolumol-org:masterfrom
eranfu:master
Apr 13, 2025
Merged

Fix the conflict with iterator_tools.#77
Luthaf merged 1 commit intolumol-org:masterfrom
eranfu:master

Conversation

@eranfu
Copy link
Contributor

@eranfu eranfu commented Apr 13, 2025

No description provided.

Copy link
Member

@Luthaf Luthaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Luthaf Luthaf merged commit 4dd8493 into lumol-org:master Apr 13, 2025
5 checks passed
@alerque
Copy link

alerque commented May 16, 2025

Does this actually make sense? How was there a conflict with a dependency that didn't even exist before this PR?

@Luthaf
Copy link
Member

Luthaf commented May 16, 2025

Both Itertools traits and SoAIndex trait where using the same function name. So if you have both in scope in code using soa_derive, rustc does not know which one to use for data.get(...), hence the fix was to use fully qualified names for calling the functions.

itertools still isn't a dependency of this project, it is only used in tests.

@alerque
Copy link

alerque commented May 16, 2025

Okay fair enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants